Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/db #66

Open
wants to merge 6,061 commits into
base: master
Choose a base branch
from
Open

Feat/db #66

wants to merge 6,061 commits into from

Conversation

Bad3r
Copy link
Owner

@Bad3r Bad3r commented Jun 16, 2023

No description provided.

logseq-cldwalker and others added 30 commits January 27, 2025 14:28
instead of harder to inspect and use d/q results
Also cleanup more more tests that have been simplified by recent test
helper improvement
for existing blocks and pages e.g. importing
Was using them across namespaces and they were needlessly different
a block data. Works within same graph currently
so that file graphs get a helpful notification that the command
can't be used instead of failing weirdly. Also remove unused
db-graph-enabled?
Add test to confirm multiple imports are idempotent.
Also fix sqlite.build not handling valid value for :db/cardinality
from properties. Also fix user specified :pages-and-blocks not building
with :build-existing-tx?
Gracefully handle invalid EDN and property conflicts
between existing and new blocks correctly. Imports
into a block worked fine because it always existed but
imports into new blocks were failing
Works across graphs for basic top-level blocks. Generalized import
command to work with block+page and other future exports
or graph. Also add tests for fresh page import
root needed to be re-rendered like with file graph import
for both existing or other graphs. Introduced an enhanced `[:build/page]` ref for
this
refs from sqlite.build as they have been replaced with
an improved [:build/page ...]
Also enhance :build/page refs so their tags export
property values. Import works well for new graphs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants